Skip to content

Fix #8 Define defaults for Java editorconfig properties#9

Open
ppalaga wants to merge 1 commit intoec4j:masterfrom
ppalaga:i8
Open

Fix #8 Define defaults for Java editorconfig properties#9
ppalaga wants to merge 1 commit intoec4j:masterfrom
ppalaga:i8

Conversation

@ppalaga
Copy link
Contributor

@ppalaga ppalaga commented Feb 3, 2019

No description provided.

@ppalaga ppalaga requested a review from dyadix February 3, 2019 21:18
@@ -0,0 +1,8 @@
root = true

[*.java]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file stores the Java defaults

StandardCharsets.UTF_8));

resourcePropertiesService = ResourcePropertiesService.builder() //
.defaultEditorConfig(javaDefaults) //
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is where the defaults from java-defaults.editorconfig are used in the JdtFormatter

root = true

[*]
charset = utf-8
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An empty .editorconfig file to test the defaults

@ppalaga
Copy link
Contributor Author

ppalaga commented Feb 3, 2019

@dyadix could you please review and eventually merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant